Conversation
|
cc @CyberShadow |
|
OK. Wasn't trying to stir anything up, just a bit confused :) |
|
Test failures due to code.dlang.org not being reachable intermediately. |
|
|
||
| auto assocArray(Range)(Range r) | ||
| if (isInputRange!Range) | ||
| if (isInputRange!Range) |
There was a problem hiding this comment.
Don't add the space back.
There was a problem hiding this comment.
It's an automatic revert & the spaces before the if constraint are the common Phobos style
There was a problem hiding this comment.
@wilzbach we've gone back and forth with the style. I think if should be flush with the previous line (no extra indent). Could you please work your magic on the autotester to make it so? :)
There was a problem hiding this comment.
we've gone back and forth with the style. I think if should be flush with the previous line (no extra indent).
Could you please work your magic on the autotester to make it so?
I will give it a try :)
wilzbach
left a comment
There was a problem hiding this comment.
Yeah it was nicer before :/
4eebf06 to
bd2f35c
Compare
|
So I rebased this to master to retrigger the CIs.
|
-> #4996 HoweverIt turns out that the changing the selective import As a consequence it was possible to use See #4981 for a fix. |
Reverts #4963